Skip to content

Conversation

alan-agius4
Copy link
Contributor

This commit updates the TypeScript configuration across the project to use moduleResolution: "bundler". This modernizes our module resolution strategy to align with current TypeScript best practices and bundler behaviors.

The following changes are included:

  • Updated tsconfig.json files to set moduleResolution to "bundler".
  • Updated the rules_angular bazel dependency to a version compatible with these changes.
  • Adjusted related test files and golden files to reflect the new module resolution strategy.

@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Sep 29, 2025
@alan-agius4 alan-agius4 added the target: major This PR is targeted for the next major release label Sep 29, 2025
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Sep 29, 2025
@alan-agius4 alan-agius4 marked this pull request as ready for review September 29, 2025 11:40
@alan-agius4 alan-agius4 requested a review from a team as a code owner September 29, 2025 11:40
@alan-agius4 alan-agius4 requested review from mmalerba, andrewseguin and devversion and removed request for a team, mmalerba and andrewseguin September 29, 2025 11:40
@alan-agius4 alan-agius4 force-pushed the module-resoluton branch 4 times, most recently from aebc4b1 to 0a477c0 Compare September 29, 2025 14:11
@alan-agius4 alan-agius4 requested a review from a team as a code owner September 29, 2025 14:11
This commit updates the TypeScript configuration across the project to use `moduleResolution: "bundler"`. This modernizes our module resolution strategy to align with current TypeScript best practices and bundler behaviors.

The following changes are included:
- Updated `tsconfig.json` files to set `moduleResolution` to `"bundler"`.
- Updated the `rules_angular` bazel dependency to a version compatible with these changes.
- Adjusted related test files and golden files to reflect the new module resolution strategy.
@alan-agius4 alan-agius4 removed the request for review from a team September 29, 2025 14:18
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Sep 29, 2025
@devversion devversion merged commit 821830d into angular:main Sep 29, 2025
24 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants